Skip to content

Conversation

@stephen-riggs
Copy link
Contributor

@stephen-riggs stephen-riggs commented Oct 15, 2025

Ability to receive messages back from tomogram picking and then send on a request to run tomogram 2d extraction/classification.

  • Registers new pj recipe for the first tomogram
  • Changes SPAFeedbackParameters to a more general ClassificationFeedbackParameters for reuse
  • Add table for tomogram picks

Currently then set to run 2D classification in batches of 10000 particles with 5 classes. Needs deciding if that is the right value.
Also probably isn't sending exactly the right set of parameters onwards to the next recipe as the recipe hasn't yet been written, but the right items should be stored in the database here.

@codecov
Copy link

codecov bot commented Oct 15, 2025

Codecov Report

❌ Patch coverage is 84.61538% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 35.77%. Comparing base (8687bba) to head (8a9f8f9).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #682      +/-   ##
==========================================
+ Coverage   35.26%   35.77%   +0.50%     
==========================================
  Files          91       92       +1     
  Lines       11225    11308      +83     
  Branches     1487     1495       +8     
==========================================
+ Hits         3959     4045      +86     
+ Misses       7138     7130       -8     
- Partials      128      133       +5     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@stephen-riggs stephen-riggs marked this pull request as ready for review October 20, 2025 11:51
Copy link
Contributor

@tieneupin tieneupin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Doesn't seem to be anything obviously wrong. Feel free to merge once you're happy with it!

@stephen-riggs stephen-riggs merged commit 443fec9 into main Oct 28, 2025
17 checks passed
@stephen-riggs stephen-riggs deleted the tomo-extract branch October 28, 2025 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants